Skip to content

feat(side-panel): improve accessibility#1857

Merged
spike-rabbit merged 1 commit intomainfrom
fix/side-panel-aria-expanded
Apr 24, 2026
Merged

feat(side-panel): improve accessibility#1857
spike-rabbit merged 1 commit intomainfrom
fix/side-panel-aria-expanded

Conversation

@akashsonune
Copy link
Copy Markdown
Member

@akashsonune akashsonune commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds the aria-expanded attribute to the side panel's collapse toggle button to improve accessibility. However, the current implementation relies on a state signal that does not account for temporary expansion modes, such as mobile drawers or hover-to-expand, which could result in inaccurate accessibility information for screen readers.

Comment thread projects/element-ng/side-panel/si-side-panel-content.component.html Outdated
@akashsonune akashsonune force-pushed the fix/side-panel-aria-expanded branch from 05ba472 to 6349dba Compare April 13, 2026 16:43
@akashsonune akashsonune marked this pull request as ready for review April 13, 2026 16:43
@akashsonune akashsonune requested review from a team as code owners April 13, 2026 16:43
@akashsonune akashsonune force-pushed the fix/side-panel-aria-expanded branch from 6349dba to fdde6f6 Compare April 14, 2026 06:31
Copy link
Copy Markdown
Member

@spike-rabbit spike-rabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you also need to add an aria-controls (see: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Attributes/aria-expanded)

Just use a generated ID for that.

@akashsonune akashsonune force-pushed the fix/side-panel-aria-expanded branch from fdde6f6 to 68ac70a Compare April 14, 2026 15:48
@akashsonune akashsonune changed the title fix(side-panel): add aria-expanded to collapse toggle button feat(side-panel): improve accessibility Apr 14, 2026
Comment thread projects/element-ng/side-panel/si-side-panel-content.component.ts Outdated
@akashsonune akashsonune force-pushed the fix/side-panel-aria-expanded branch from 68ac70a to 617a91a Compare April 15, 2026 11:09
@akashsonune akashsonune requested a review from dr-itz April 15, 2026 11:09
Copy link
Copy Markdown
Member

@spike-rabbit spike-rabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@spike-rabbit spike-rabbit force-pushed the fix/side-panel-aria-expanded branch from 617a91a to b569573 Compare April 24, 2026 09:13
@spike-rabbit spike-rabbit enabled auto-merge April 24, 2026 09:13
@spike-rabbit spike-rabbit added this pull request to the merge queue Apr 24, 2026
Merged via the queue into main with commit e6dd452 Apr 24, 2026
11 checks passed
@spike-rabbit spike-rabbit deleted the fix/side-panel-aria-expanded branch April 24, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants